Skip to content

Add clang plugin for class usage replacing (::std::atomic<T> to LTestAtomic<T>)#14

Draft
dmitrii-artuhov wants to merge 47 commits intoITMO-PTDC-Team:masterfrom
dmitrii-artuhov:dartiukhov/feature/clang-pass
Draft

Add clang plugin for class usage replacing (::std::atomic<T> to LTestAtomic<T>)#14
dmitrii-artuhov wants to merge 47 commits intoITMO-PTDC-Team:masterfrom
dmitrii-artuhov:dartiukhov/feature/clang-pass

Conversation

@dmitrii-artuhov
Copy link
Copy Markdown
Contributor

@dmitrii-artuhov dmitrii-artuhov commented Mar 27, 2025

Можно смотреть после мержа этого ПРа и ребейза на него, либо наоборот ДО.

ПР имплементирует следующий функционал:

  • Заменяет вхождения ::std::atomic<T> на кастомный класс LTestAtomic<T>

TODO:

  • сделать ребейз на мастер
  • пофиксить dockerfile (добавив скачивание clang зависимостей)
  • пофиксить билд на CI (также не хватает зависимостей для clang)
  • закончить апи LTestAtomic<T>, чтобы оно совпадало с атомиками (есть одна проблема с специализацией T*, как ее решить без жесткой копи-пасты не знаю. Думаю, не критично, это можно добавить позже
  • Optional: Сделать в cmake более явную разницу между clang- & llvm-pass make (т.е. сделать имена LLVM_PASS, CLANG_PASS, CLANG_TOOL)

@dmitrii-artuhov dmitrii-artuhov force-pushed the dartiukhov/feature/clang-pass branch from 58933f6 to f2b8044 Compare April 1, 2025 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant